Skip to content

B-side LLM-only memory extractors - #6

Closed
fym2001 wants to merge 3 commits into
DCC-Saber:masterfrom
fym2001:feat/b-llm-only-memory-extractors
Closed

B-side LLM-only memory extractors#6
fym2001 wants to merge 3 commits into
DCC-Saber:masterfrom
fym2001:feat/b-llm-only-memory-extractors

Conversation

@fym2001

@fym2001 fym2001 commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Replace B-side hardcoded/rule-based memory extraction with LLM-only semantic extraction.
  • Keep existing public extractor class names and method signatures compatible with MemoryEvent -> MemoryCandidate.
  • Keep local logic limited to prompt packaging, sensitive-data redaction, JSON-to-candidate conversion, validation, deduplication, and conflict annotation.
  • Add demo and documentation for before/after dataset input-output display.

Verification

python -m pytest tests/test_extractors.py tests/test_llm_memory_extractor.py tests/test_environment_extractor.py tests/test_tool_extractor.py tests/test_workflow_extractor.py tests/test_ingestion_to_extractors.py -q

Result: 30 passed

python demo\b_llm_pipeline_demo.py

Demo prints raw MemoryEvent[], sanitized LLM input JSON, fake LLM JSON output, and final MemoryCandidate[].

Note

This PR supersedes the previous B LLM pipeline PR. The older PR should not be used for review.

@fym2001

fym2001 commented Aug 2, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by #7, which contains the complete LLM-only extraction pipeline plus the new MemoryCandidate-to-MemoryRecord admission workflow. Please review #7 instead.

@fym2001 fym2001 closed this Aug 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant